Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Puppeteer E2E test: Show actual and difference screenshots #25410

Merged
merged 4 commits into from
Feb 6, 2023

Conversation

LeviPesin
Copy link
Contributor

@LeviPesin LeviPesin commented Feb 2, 2023

Related issue: #24109

Description

Make Puppeteer show actual and difference screenshots. This is useful while debugging to "understand" what Puppeteer job running on GitHub Actions really sees, and how it differs from the screenshot in examples/screenshots or a local screenshot.

@LeviPesin LeviPesin changed the title Puppeteer E2E test: Show actual and diff screenshots Puppeteer E2E test: Show actual and difference screenshots Feb 2, 2023
test/e2e/puppeteer.js Fixed Show fixed Hide fixed
test/e2e/puppeteer.js Fixed Show fixed Hide fixed
@mrdoob
Copy link
Owner

mrdoob commented Feb 3, 2023

Hmm, where will these images be hosted?

@LeviPesin
Copy link
Contributor Author

On GitHub Actions for 1-3 months (depending on settings of the repository) -- I think for public repositories it's free.

@LeviPesin
Copy link
Contributor Author

LeviPesin commented Feb 3, 2023

Currently no artifacts is uploaded because there are no failing files.

@mrdoob mrdoob added this to the r150 milestone Feb 6, 2023
@mrdoob mrdoob merged commit 61a06c4 into mrdoob:dev Feb 6, 2023
@LeviPesin LeviPesin deleted the puppeteer-show-actual-screenshots branch February 6, 2023 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants